Skip to content

fix(ci): retire scorecard-enforcer.yml, adopt canonical scorecard.yml#159

Merged
hyperpolymath merged 3 commits into
mainfrom
fix/retire-scorecard-enforcer
Jul 1, 2026
Merged

fix(ci): retire scorecard-enforcer.yml, adopt canonical scorecard.yml#159
hyperpolymath merged 3 commits into
mainfrom
fix/retire-scorecard-enforcer

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

scorecard-enforcer.yml is retired estate-wide (governance's check-workflow-staleness.sh flags its presence, and OSSF Scorecard must not upload SARIF to Code Scanning unless it runs for every PR head commit — the enforcer's push+schedule-only trigger doesn't). Replaced with the canonical scorecard.yml → standards scorecard-reusable.yml pattern (matches what other repos already run).

…t fix

Pinned SHA predated standards#439 (fix(rust-ci): pass explicit
toolchain: stable to SHA-pinned dtolnay action), so every run failed
at 'toolchain is a required input' before any real check executed.
Re-pinned to standards main tip (7c9db0e5), which is at/after #439.
scorecard-enforcer.yml is retired estate-wide (governance's
check-workflow-staleness.sh flags its presence, and OSSF Scorecard
must not upload SARIF to Code Scanning unless it runs for every PR
head commit -- the enforcer's push+schedule-only trigger doesn't).
Replaced with the canonical scorecard.yml -> standards
scorecard-reusable.yml pattern.
@hyperpolymath
hyperpolymath enabled auto-merge (squash) July 1, 2026 11:49
Signed-off-by: Jonathan D.A. Jewell <6759885+hyperpolymath@users.noreply.github.com>
@hyperpolymath
hyperpolymath merged commit 65fec4a into main Jul 1, 2026
3 checks passed
@hyperpolymath
hyperpolymath deleted the fix/retire-scorecard-enforcer branch July 1, 2026 11:53
@sonarqubecloud

sonarqubecloud Bot commented Jul 1, 2026

Copy link
Copy Markdown

hyperpolymath added a commit that referenced this pull request Jul 1, 2026
…F→Code Scanning) (#162)

## Summary

Fixes the **recurring `governance / Check Workflow Staleness` red** that
has failed on
every recent PR (#154, #161, …).

**Root cause (pre-existing, from #159):** the Scorecard workflow adopted
in #159 runs
Scorecard in **SARIF** mode and pushes the SARIF into **GitHub Code
Scanning**, but only
fires on `push(main)`/`schedule` — not per-PR-head. The standards
governance staleness
gate forbids exactly that:

> `::error:: OSSF Scorecard must not upload SARIF to GitHub Code
Scanning unless it runs
> for every PR head commit.`

**Fix:** delegate `scorecard.yml` to the canonical standards
`scorecard-reusable.yml@7c9db0e` — it runs Scorecard in **JSON** mode
with
`publish_results` and uploads an **artifact**, and does **not** push
SARIF into Code
Scanning. That's the canonical behaviour #159's title ("adopt canonical
scorecard.yml")
intended, and it satisfies the gate. Job name (`analysis`) preserved for
the required-check
contract.

Also hardened `must-check.sh` (portable `[[:space:]]` instead of
GNU-only `\s`; quoted
`exit "$fail"`).

**Verified locally:** `check-workflow-staleness.sh` → "All workflow
staleness checks
passed"; `scorecard.yml` YAML valid; `must-check` exit 0.

## FLAGS (unchanged)
- `gossamer` + `conative-gating` still 403-blocked → #83 / #103 staged
(needs the repos
  added to the environment's scope).
- OPEN proof obligations 1.1 / 1.2 (formal) / 3.2 remain honestly
staged.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

https://claude.ai/code/session_0172RBMz3qYjb1ttzD2i7RNh

---
_Generated by [Claude
Code](https://claude.ai/code/session_0172RBMz3qYjb1ttzD2i7RNh)_

Co-authored-by: Claude <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant